-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added synthetic beans for the managed ExecutorService backed by virtual threads #36248
Added synthetic beans for the managed ExecutorService backed by virtual threads #36248
Conversation
ozangunalp
commented
Oct 2, 2023
•
edited
Loading
edited
- Shutdown operations on the VT ThreadPerTask Executor throws runtime exception
- Simplified the Thread per Task Executor creation
- Configured the UncaughtExceptionHandler
- Added a section to VT guide for injecting the ExecutorService
3044b38
to
d0c67fc
Compare
.../runtime/src/main/java/io/quarkus/virtual/threads/FallbackVirtualThreadsExecutorService.java
Outdated
Show resolved
Hide resolved
fa2e4d7
to
58f1cdf
Compare
...tual-threads/runtime/src/main/java/io/quarkus/virtual/threads/DelegatingExecutorService.java
Outdated
Show resolved
Hide resolved
.../runtime/src/main/java/io/quarkus/virtual/threads/FallbackVirtualThreadsExecutorService.java
Outdated
Show resolved
Hide resolved
...virtual-threads/runtime/src/main/java/io/quarkus/virtual/threads/VirtualThreadsRecorder.java
Show resolved
Hide resolved
58f1cdf
to
18a7028
Compare
This comment has been minimized.
This comment has been minimized.
extensions/virtual-threads/runtime/src/main/java/io/quarkus/virtual/threads/VirtualThreads.java
Outdated
Show resolved
Hide resolved
18a7028
to
93c787c
Compare
This comment has been minimized.
This comment has been minimized.
93c787c
to
0430cd9
Compare
This comment has been minimized.
This comment has been minimized.
...ual-threads/deployment/src/main/java/io/quarkus/virtual/threads/VirtualThreadsProcessor.java
Outdated
Show resolved
Hide resolved
0430cd9
to
4877450
Compare
🙈 The PR is closed and the preview is expired. |
This comment has been minimized.
This comment has been minimized.
@ozangunalp can you rebase so we know if the CI is going to be insane again? |
…xecutorService backed by virtual threads
4877450
to
d5ba035
Compare
Failing Jobs - Building d5ba035
Full information is available in the Build summary check run. Failures⚙️ JVM Tests - JDK 11 #- Failing: extensions/oidc-client-graphql/deployment
📦 extensions/oidc-client-graphql/deployment✖
⚙️ JVM Tests - JDK 17 #- Failing: extensions/oidc-client-graphql/deployment
📦 extensions/oidc-client-graphql/deployment✖
⚙️ JVM Tests - JDK 20 #- Failing: extensions/oidc-client-graphql/deployment
📦 extensions/oidc-client-graphql/deployment✖
⚙️ Maven Tests - JDK 11 Windows #- Failing: integration-tests/maven
📦 integration-tests/maven✖
✖
✖
✖
|
Ok, the CI issues are back to the regular ones. |